-
-
Notifications
You must be signed in to change notification settings - Fork 398
Replace ctx.ToolsDirs with ctx.BuiltInToolsDirs #418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The ToolsDirs field has been replaced by BuiltInToolsDirs during the big refactoring in the arduino-builder repo. The intention was to remove ToolsDirs once done, but for some reasons I forgot to remove it. This commit completes the transition.
dd9d1d7
to
b677403
Compare
@ArduinoBot build this please |
✅ Build completed. ⬇️ Linux 64: ⬇️ Linux 32: ⬇️ Linux ARM 32: ⬇️ Linux ARM 64: ⬇️ Windows: ⬇️ OSX: ℹ️ To test this build:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This will include the following upstream fixes: arduino/arduino-cli#418 arduino/arduino-cli#422 arduino/arduino-cli#428 arduino/arduino-cli#429
This will include the following upstream fixes: arduino/arduino-cli#418 arduino/arduino-cli#422 arduino/arduino-cli#428 arduino/arduino-cli#429
The ToolsDirs field has been replaced by BuiltInToolsDirs during the big refactoring in the arduino-builder repo.
The intention was to remove ToolsDirs once done, but for some reasons I forgot to remove it.
This commit completes the transition.